-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dataset Publication Tutorial #711
Conversation
for more information, see https://pre-commit.ci
…thoutBorders/nwb-guide into publication-tutorial
for more information, see https://pre-commit.ci
Ah, but the constant commiting of the snapshots on chain PRs can produce conflicts... I'm assuming that is only an annoyance for the current initial push on these and won't occur unintentionally in the future |
This PR also has build errors on the sphinx side, various unknown references |
Yeah, we coordinated about this after I'd pushed here. I've overridden the conflicted files with the originals from the base branch. |
@garrettmflynn Failing tests now |
Co-authored-by: Cody Baker <[email protected]>
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
…thoutBorders/nwb-guide into publication-tutorial
for more information, see https://pre-commit.ci
…thoutBorders/nwb-guide into publication-tutorial
for more information, see https://pre-commit.ci
Co-authored-by: Cody Baker <[email protected]>
Looks great! |
This PR implements the dataset publication workflow for the tutorials, which modifies the Multi-Session workflow to enable publishing on DANDI and completes a full upload.